Skip to content

Add preselect to subscribe page#1111

Open
TatevikGr wants to merge 4 commits into
release-3.7.0from
pre-select-list-in-subscribe-page
Open

Add preselect to subscribe page#1111
TatevikGr wants to merge 4 commits into
release-3.7.0from
pre-select-list-in-subscribe-page

Conversation

@TatevikGr
Copy link
Copy Markdown

@TatevikGr TatevikGr commented May 12, 2026

Description

Add preselect to the subscribe page: a list can be marked as preselected when creating a public subscribe page.

Related Issue

issue

Screenshots (if appropriate):

image

@TatevikGr TatevikGr force-pushed the pre-select-list-in-subscribe-page branch from 408bf49 to 800ec4f Compare May 14, 2026 12:17
@TatevikGr TatevikGr requested a review from bramley May 18, 2026 05:53
@bramley
Copy link
Copy Markdown
Contributor

bramley commented May 19, 2026

This change works in my testing, but I think that the user interface is unclear. I think a tabular structure will be clearer, and have created this example. There is one row for each list and a row at the end for the "none" checkbox. It also has explanatory text for the default. I think using "default" (or maybe something else) is clearer than "pre-select" because "select" is already being used for choosing the lists to include.

image

@TatevikGr
Copy link
Copy Markdown
Author

TatevikGr commented May 19, 2026

I think a tabular structure will be clearer, and have created this example. There is one row for each list and a row at the end for the "none" checkbox. It also has explanatory text for the default. I think using "default" (or maybe something else) is clearer than "pre-select" because "select" is already being used for choosing the lists to include.

Yes, your version is definitely better, updated

@TatevikGr TatevikGr force-pushed the pre-select-list-in-subscribe-page branch from 5b46b00 to 5cd0f03 Compare May 19, 2026 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants